refacto: migration de l'administration des articles sous Symfony #1893
Conversation
vgreb
force-pushed
the
refacto/articles
branch
4 times, most recently
from
9203f65 to
767d4a9
Compare
| @@ -38,3 +38,9 @@ | |||
| </div> | |||
| {%- endif -%} | |||
| {%- endblock datetime_widget -%} | |||
|
|
|||
| {%- block textarea_widget -%} | |||
There was a problem hiding this comment.
Il est peut être plus judicieux d'utiliser la macro wysiwyg() présente dans templates/admin/event/form.html.twig, quite à la déplacer dans le form_theme_admin.html.twig.
Qu'en pensez-vous ?
There was a problem hiding this comment.
Si ça rend le code plus clair pourquoi pas.
Attention par contre à si ça modifie beaucoup de fichiers, ce sera mieux de faire ça dans une PR séparée.
vgreb
changed the title
sources/AppBundle/Controller/Admin/Site/Article/EditArticleAction.php
Outdated
Show resolved
Hide resolved
sources/AppBundle/Controller/Admin/Site/Article/EditArticleAction.php
Outdated
Show resolved
Hide resolved
sources/AppBundle/Controller/Admin/Site/Article/EditArticleAction.php
Outdated
Show resolved
Hide resolved
sources/AppBundle/Controller/Admin/Site/Article/AddArticleAction.php
Outdated
Show resolved
Hide resolved
vgreb
force-pushed
the
refacto/articles
branch
from
9b338cd to
fa7f862
Compare
vgreb
force-pushed
the
refacto/articles
branch
from
fa7f862 to
78ea080
Compare
|
J'ai checkout la branche en local et trouvé un soucis. Quand j'ouvre un article existant, le markdown est transformé en HTML dans l'éditeur. Les boutons de l'éditeur par contre font bien du markdown, mais il n'est pas rendu en front. Sur la prod : Le même article avec ta branche : Si besoin je peux te fournir du SQL de prod pour cet article, dis moi sur Slack. |
| 'label' => 'Chapeau', | ||
| 'required' => false, | ||
| 'attr' => [ | ||
| 'maxlength' => 255, |
There was a problem hiding this comment.
En base actuellement le chapeau le plus long est de 836 caractères.
La plupart des anciens articles ont entre 200 et 500 caractères et ne sont donc plus modifiables.
On peut se dire 1000 pour être tranquilles.
|
Je viens de vérifier et en prod il y a 768 anciens articles en HTML. Les plus récents en markdown sont environ 200. Je serais d'avis de pas s'embêter à gérer la modification des anciens, donc de bien interpréter les articles comme du markdown. @agallou et/ou @stakovicz ça vous parait raisonnable ? |
Migration de l'administration de articles sous Symfony.
La nouvelle url dans le menu est /admin/site/articles
J'ai volontairement laissé les sources legacy. Si la MR est validée, je ferais une seconde MR pour les supprimer.